Skip to content

dev-qemu: Add task for testing PCC/eSPI - #48

Draft
kurtjd wants to merge 1 commit into
OpenDevicePartnership:mainfrom
kurtjd:espi
Draft

kurtjd wants to merge 1 commit into
OpenDevicePartnership:mainfrom
kurtjd:espi

Conversation

@kurtjd

@kurtjd kurtjd commented Sep 29, 2026 •

Copy link
Copy Markdown
Member

This is intended to exercise the new virtual eSPI/PCC addition to qemu for our custom ec machine.

A separate change to the ec test-cli app allows for the app to send a basic command + payload (in this case, just a PING command with an arbitrary number) through the ectest driver through the Windows PCC interface through the virtual eSPI bus (over peripheral channel) to the EC (which is processed by this task).

This task then replies PONG by updating the shared memory, notifying the host via vwire, and the host then sends a peripheral channel read (which gets handled behind the scenes by qemu).

Right now only really testing PCC type 3, type 4 functionality might be added later when we actually need it (though type 4 is supported in qemu).

Draft until a few dependent PRs on other repos are merged.

@kurtjd kurtjd self-assigned this Sep 29, 2026
Copilot AI balanced review requested due to automatic review settings September 29, 2026 19:56
@kurtjd
kurtjd requested a review from a team as a code owner September 29, 2026 19:56
@kurtjd kurtjd added this to EC Sep 29, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@kurtjd

kurtjd commented Oct 2, 2026 •

Copy link
Copy Markdown
Member Author

Note: I'll need to update the pinned revision for odp-qemu-builder and odp-platform-common in check.yml after OpenDevicePartnership/odp-qemu-builder#10 and OpenDevicePartnership/odp-platform-common#221 are merged for CI to pass. Draft until this is done.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants